Skip to content

chore(eql): repoint manifests at cipherstash/stack - #937

Merged
tobyhede merged 1 commit into
mainfrom
chore/eql-repoint-manifests-to-stack
Aug 21, 2026
Merged

chore(eql): repoint manifests at cipherstash/stack#937
tobyhede merged 1 commit into
mainfrom
chore/eql-repoint-manifests-to-stack

Conversation

@tobyhede

Copy link
Copy Markdown
Contributor

Summary

Trusted publishing for @cipherstash/eql (npm) and eql-bindings (crates.io) is now configured against cipherstash/stack, but the manifests themselves still named cipherstash/encrypt-query-language. Both npm and crates.io reject a publish whose manifest repository field doesn't match the publishing repository, so a release triggered from here today would fail. This is the last blocker before EQL can actually release from cipherstash/stack — the "trusted publishing points at stack" step alone isn't the same as "a release can succeed here."

Changes

  • packages/eql/packages/eql/package.json: repository.urlgit+https://github.com/cipherstash/stack.git, repository.directorypackages/eql/packages/eql (the npm package's real location under the verbatim-prefix subtree import), bugs.urlhttps://github.com/cipherstash/stack/issues.
  • packages/eql/crates/eql-bindings/Cargo.toml: repository and homepagehttps://github.com/cipherstash/stack.
  • Added scripts/__tests__/eql-repository-urls.test.mjs, mirroring ffi-repository-urls.test.mjs, so a regression here fails CI instead of a future publish.
  • Checked release-plz.toml's prose — it doesn't reference the old repository, so it needed no change.

Verification

  • New guard test passes: npx vitest run --config scripts/vitest.config.mjs scripts/__tests__/eql-repository-urls.test.mjs — 5/5.
  • pnpm run code:fix (Biome) run scoped to the changed files — no other issues.
  • Three unrelated test files fail both with and without this change (lint-no-eql-registry-pins.test.mjs, cargo-lock-freshness.test.mjs, bench-index-expressions.test.mjs) — confirmed pre-existing on main via git stash, not caused by this PR.
  • Did not run a full package build; this is a metadata-only change with no code paths reading these fields at runtime.

Related

Part of the EQL → cipherstash/stack trusted-publishing cutover.

npm and crates.io both reject a publish whose manifest repository
field doesn't match the publishing repository. Trusted publishing for
@cipherstash/eql and eql-bindings is now configured against
cipherstash/stack, but the manifests still named
cipherstash/encrypt-query-language — so a release triggered from here
would fail regardless. Repoints package.json (repository.url,
repository.directory, bugs.url) and eql-bindings' Cargo.toml
(repository, homepage), and adds a guard test mirroring
ffi-repository-urls.test.mjs so this can't regress silently.

release-plz.toml's prose was checked and needs no change — it doesn't
reference the old repository.
@changeset-bot

changeset-bot Bot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e52d331

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 12 packages
Name Type
@cipherstash/eql Patch
stash Patch
@cipherstash/stack-prisma Patch
@cipherstash/basic-example Patch
@cipherstash/e2e Patch
@cipherstash/prisma-example Patch
@cipherstash/stack Patch
@cipherstash/stack-drizzle Patch
@cipherstash/stack-supabase Patch
@cipherstash/wizard Patch
@cipherstash/bench Patch
@cipherstash/test-kit Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@tobyhede
tobyhede marked this pull request as ready for review August 21, 2026 03:47
@tobyhede
tobyhede requested a review from a team as a code owner August 21, 2026 03:47
@tobyhede
tobyhede requested a review from freshtonic August 21, 2026 03:48
@tobyhede
tobyhede merged commit ca73000 into main Aug 21, 2026
37 checks passed
@tobyhede
tobyhede deleted the chore/eql-repoint-manifests-to-stack branch August 21, 2026 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants